[#200] ProfileView에 분기 활동을 주 단위로 그래프를 통해 보여주도록 구현한다#208
Conversation
opficdev
commented
Mar 6, 2026
opficdev
commented
- closed ProfileView에 분기 활동을 주 단위로 그래프를 통해 보여주도록 구현한다 #200
Summary of ChangesHello, I'm Gemini Code Assist1! I'm currently reviewing this pull request and will post my feedback shortly. In the meantime, here's a summary to help you and other reviewers quickly get up to speed! 이 PR은 Highlights
Changelog
Using Gemini Code AssistThe full guide for Gemini Code Assist can be found on our documentation page, here are some quick tips. Invoking Gemini You can request assistance from Gemini at any point by creating a comment using either
Customization To customize Gemini Code Assist for GitHub experience, repository maintainers can create a configuration file and/or provide a custom code review style guide (such as PEP-8 for Python) by creating and adding files to a Limitations & Feedback Gemini Code Assist may make mistakes. Please leave feedback on any instances where its feedback is incorrect or counter productive. You can react with 👍 and 👎 on @gemini-code-assist comments. If you're interested in giving your feedback about your experience with Gemini Code Assist for Github and other Google products, sign up here. You can also get AI-powered code generation, chat, as well as code reviews directly in the IDE at no cost with the Gemini Code Assist IDE Extension. Footnotes
|
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
| let year = calendar.component(.year, from: start) | ||
| let month = calendar.component(.month, from: start) | ||
| let quarter = ((month - 1) / 3) + 1 | ||
| return "\(year) Q\(quarter)" |
There was a problem hiding this comment.
분기(Quarter)를 나타내는 문자열 "Q"가 하드코딩되어 있습니다. Localizable.xcstrings에 Q%lld 키가 추가되었으므로, 이를 활용하여 지역화(Localization)를 지원하도록 수정하는 것이 좋습니다. 이렇게 하면 다른 언어로 번역이 필요할 때 쉽게 대응할 수 있습니다.
| return "\(year) Q\(quarter)" | |
| return "\(year) \(String(format: NSLocalizedString("Q%lld", comment: ""), quarter))" |
| .foregroundStyle(.quaternary) | ||
| AxisValueLabel { | ||
| if let weekIndex = value.as(Int.self) { | ||
| Text("\(weekIndex)주") |
There was a problem hiding this comment.
X축 레이블에 사용된 '주'라는 문자열이 하드코딩되어 있습니다. Localizable.xcstrings에 "%lld주" 키가 있으므로 이를 사용하여 지역화 처리를 하는 것이 좋습니다. 이렇게 하면 유지보수성과 다국어 지원에 유리합니다.
| Text("\(weekIndex)주") | |
| Text(String(format: NSLocalizedString("%lld주", comment: ""), weekIndex)) |
| private var createdSeries: some ChartContent { | ||
| ForEach(trendPoints) { point in | ||
| LineMark( | ||
| x: .value("주차", point.weekIndex), |
There was a problem hiding this comment.
차트의 X축 값에 대한 설명으로 "주차" 문자열이 하드코딩되어 있습니다. 이 문자열은 사용자에게 직접 보이지 않을 수 있지만, 디버깅이나 향후 차트 라이브러리 변경 시 중요할 수 있습니다. Localizable.xcstrings에 있는 "주차" 키를 사용하여 지역화하는 것을 권장합니다. 이 피드백은 completedSeries에도 동일하게 적용됩니다.
| x: .value("주차", point.weekIndex), | |
| x: .value(NSLocalizedString("주차", comment: ""), point.weekIndex), |
| set: { viewModel.send(.setQuarterPickerYear($0)) } | ||
| )) { | ||
| ForEach(viewModel.availableQuarterYears, id: \.self) { year in | ||
| Text(year.formatted(.number.grouping(.never)) + "년").tag(year) |
There was a problem hiding this comment.
연도를 표시하는 부분에서 '년'이 하드코딩되어 있습니다. 다국어 지원 및 유지보수를 위해 Localizable.xcstrings에 "%@년"과 같은 형식의 키를 추가하고 이를 코드에서 사용하는 것을 권장합니다.
| Text(year.formatted(.number.grouping(.never)) + "년").tag(year) | |
| Text("\(year.formatted(.number.grouping(.never)))년").tag(year) // Localizable.xcstrings에 "%@년" 키를 추가하고 NSLocalizedString을 사용하도록 수정 제안 |
| guard let quarterStart else { return } | ||
| viewModel.send(.selectQuarter(quarterStart)) | ||
| } label: { | ||
| Text("Q\(quarter)") |
There was a problem hiding this comment.
분기 선택 버튼의 라벨에 "Q"가 하드코딩되어 있습니다. Localizable.xcstrings에 Q%lld 키가 정의되어 있으므로, 이를 사용하여 지역화된 문자열을 생성하는 것이 좋습니다.
| Text("Q\(quarter)") | |
| Text(String(format: NSLocalizedString("Q%lld", comment: ""), quarter)) |
1 participant
